Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
• If you have done the above and are still having issues with the CLA being reported as unsigned, please email the CNCF helpdesk: helpdesk@rt.linuxfoundation.org

Signed CLA

/assign @justinsb

Codecov Report

Merging #2438 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2438   +/-   ##
=======================================
  Coverage   41.37%   41.37%           
=======================================
  Files          74       74           
  Lines        5262     5262           
=======================================
  Hits         2177     2177           
  Misses       2789     2789           
  Partials      296      296

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22f7832...ce9d42b. Read the comment docs.

/approve
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aledbf, SamFleming

The full list of commands accepted by this bot can be found here.

The pull request process is described here

@SamFleming thanks!